Skip to content

Comments

스터디 시작일 기준 모집 마감 처리 추가#391

Merged
HA-SEUNG-JEONG merged 1 commit intodevelopfrom
bug/group-study
Feb 11, 2026
Merged

스터디 시작일 기준 모집 마감 처리 추가#391
HA-SEUNG-JEONG merged 1 commit intodevelopfrom
bug/group-study

Conversation

@HA-SEUNG-JEONG
Copy link
Contributor

@HA-SEUNG-JEONG HA-SEUNG-JEONG commented Feb 11, 2026

🌱 연관된 이슈

☘️ 작업 내용

스터디 시작일이 오늘 이전이거나 같은 경우 신청 버튼을 비활성화하고
'모집 마감' 텍스트를 표시하도록 처리

🍀 참고사항

스크린샷 (선택)

Summary by CodeRabbit

릴리스 노트

  • 새로운 기능
    • 스터디 모집 마감 시간 확인 기능 추가: 모집 기한이 지난 경우 지원 버튼이 비활성화되고 "모집 마감" 텍스트로 표시됩니다.

스터디 시작일이 오늘 이전이거나 같은 경우 신청 버튼을 비활성화하고
'모집 마감' 텍스트를 표시하도록 처리

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@HA-SEUNG-JEONG HA-SEUNG-JEONG self-assigned this Feb 11, 2026
@vercel
Copy link

vercel bot commented Feb 11, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
study-platform-client-dev Ready Ready Preview, Comment Feb 11, 2026 3:22am

@coderabbitai
Copy link

coderabbitai bot commented Feb 11, 2026

📝 Walkthrough

Walkthrough

학습 시작 날짜와 현재 날짜를 비교하여 모집 마감 여부를 판단하는 데드라인 체크 로직을 추가했습니다. 이를 통해 마감 기한이 지난 경우 신청 버튼을 비활성화하고 버튼 텍스트를 "모집 마감"으로 표시합니다.

Changes

Cohort / File(s) Summary
Deadline 체크 로직 추가
src/components/summary/study-info-summary.tsx
startDate와 현재 날짜 비교를 통한 isDeadlinePassed 플래그 도입. 버튼 비활성화 조건 및 텍스트 레이블링에 데드라인 상태 반영.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐰 마감일을 재빠르게 확인하고,
모집이 닫혔는지 슬기롭게 파악하지!
버튼은 쉬고, 텍스트는 춤을 추네,
"모집 마감" 아홉 줄의 우아함 ✨

🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed Pull request 제목이 변경사항의 핵심을 명확하게 설명하고 있습니다. 스터디 시작일을 기준으로 모집 마감 처리를 추가하는 것이 주요 변경사항이며, 제목이 이를 정확하게 반영하고 있습니다.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch bug/group-study

Tip

Issue Planner is now in beta. Read the docs and try it out! Share your feedback on Discord.


Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)
src/components/summary/study-info-summary.tsx (1)

239-246: ⚠️ Potential issue | 🟡 Minor

비로그인 상태에서도 ‘모집 마감’이 반영되어야 합니다.
현재는 마감이어도 “신청하기”가 노출되고 로그인으로 이동됩니다. 마감 시 비활성화 + 텍스트 변경이 필요합니다.

🔧 제안 수정
          <Button
            size="large"
            color="primary"
            className="h-600"
            onClick={handleApplyClick}
+           disabled={isDeadlinePassed}
          >
-           신청하기
+           {isDeadlinePassed ? '모집 마감' : '신청하기'}
          </Button>

@HA-SEUNG-JEONG HA-SEUNG-JEONG merged commit dfb3282 into develop Feb 11, 2026
11 checks passed
@HA-SEUNG-JEONG HA-SEUNG-JEONG deleted the bug/group-study branch February 11, 2026 03:41
@HA-SEUNG-JEONG HA-SEUNG-JEONG restored the bug/group-study branch February 11, 2026 03:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant